Skip to content

OSAC-1675: Add e2e-test environment and required status checks to component repos#109

Merged
eliorerz merged 2 commits into
osac-project:mainfrom
eliorerz:OSAC-1675-shared-e2e-environments
Jun 30, 2026
Merged

OSAC-1675: Add e2e-test environment and required status checks to component repos#109
eliorerz merged 2 commits into
osac-project:mainfrom
eliorerz:OSAC-1675-shared-e2e-environments

Conversation

@eliorerz

@eliorerz eliorerz commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add e2e-test GitHub environment to fulfillment-service, osac-operator, and osac-aap (gates fork PRs from accessing self-hosted runners via environment protection rules)
  • Add required_status_checks to enforce merge gating:
    • fulfillment-service: ci/prow/unit, e2e-full-install / e2e
    • osac-operator: ci/prow/temp, e2e-full-install / e2e
    • osac-aap: ci/prow/temp, e2e-full-install / e2e

Test plan

  • Verify Terraform plan shows expected environment and branch protection changes
  • Confirm e2e-test environment appears in each repo's Settings → Environments after apply

@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@eliorerz, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 34 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Repository: osac-project/coderabbit/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Enterprise

Run ID: d2fdd22d-2ef6-4114-8e55-ec3342c59ac1

📥 Commits

Reviewing files that changed from the base of the PR and between c54e711 and 812df6a.

📒 Files selected for processing (1)
  • repositories.tf
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@eliorerz eliorerz force-pushed the OSAC-1675-shared-e2e-environments branch from 5c4fab4 to d4891fb Compare June 22, 2026 18:22
@eliorerz eliorerz force-pushed the OSAC-1675-shared-e2e-environments branch from d4891fb to 5a650dd Compare June 30, 2026 01:16
@eliorerz eliorerz changed the title OSAC-1675: Add e2e-test environment to component repos OSAC-1675: Add e2e-test environment and required status checks to component repos Jun 30, 2026
@eliorerz eliorerz force-pushed the OSAC-1675-shared-e2e-environments branch from 5a650dd to 91b5027 Compare June 30, 2026 01:33
Add the e2e-test GitHub environment to fulfillment-service,
osac-operator, osac-aap, and osac-installer. This gates fork PRs
from accessing self-hosted runners via environment protection rules.
@eliorerz eliorerz force-pushed the OSAC-1675-shared-e2e-environments branch from 91b5027 to ad5192d Compare June 30, 2026 01:56
Rename required check from "e2e-full-install / e2e" to
"e2e-vmaas-full-install / e2e" for fulfillment-service,
osac-operator, osac-aap, and osac-installer. This matches the
job key rename in each repo's caller workflow.
@eliorerz eliorerz merged commit 51dd648 into osac-project:main Jun 30, 2026
2 checks passed
@eliorerz eliorerz deleted the OSAC-1675-shared-e2e-environments branch June 30, 2026 03:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant